Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MIT license #37

Merged
merged 2 commits into from
Jan 23, 2016
Merged

Add MIT license #37

merged 2 commits into from
Jan 23, 2016

Conversation

todoubled
Copy link
Contributor

I'd be more comfortable using this if it had a license. Are there any objections to MIT, chosen from http://choosealicense.com/?

Thanks for maintaining this lib!

@sieira
Copy link
Collaborator

sieira commented Jan 19, 2016

Unfortunately, I can't do this without the consent of the original maintainers, I'll give it a look tomorrow. I'd like to license it as well, avoids a lot of trouble

@briankircho
Copy link
Owner

I am the original maintainer, this is fine with me I usually go with MIT guess I just forgot to add it

@todoubled
Copy link
Contributor Author

Great, thanks guys. I updated this to use your full name @briankircho, so it should be good to go.

sieira added a commit that referenced this pull request Jan 23, 2016
@sieira sieira merged commit aeb7173 into briankircho:master Jan 23, 2016
@sieira
Copy link
Collaborator

sieira commented Jan 23, 2016

Ok, merged, thanks :)

@todoubled todoubled deleted the add-mit-license branch January 25, 2016 16:51
@todoubled
Copy link
Contributor Author

Thanks @sieira! Would it be possible to get a new version released to include the license too?

@sieira
Copy link
Collaborator

sieira commented Feb 9, 2016

It's done, I hope there are no regressions (proxies may be a problem for now)

@todoubled
Copy link
Contributor Author

@sieira Thanks! Can you please elaborate on the potential proxies problem? I'm not quite sure what that refers to.

@sieira
Copy link
Collaborator

sieira commented Feb 10, 2016

See my last comment here #39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants